-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: restructure deploy doc #567
Conversation
content/docs/faq.md
Outdated
Their entry is removed from the `project.yaml` file and the associated volumes are deleted. This validation ensures that volumes exist for current projects only and that volumes are not reused between different projects of the same name. | ||
|
||
## 12. How do I deploy my application to a local Minikube cluster? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this title is right. AFAIK, appsody can deploy to minikube fine - no special config should be needed. Please can you confirm this once?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The intent was to document how to use Docker daemon from minikube vs Docker on host.
content/docs/faq.md
Outdated
Their entry is removed from the `project.yaml` file and the associated volumes are deleted. This validation ensures that volumes exist for current projects only and that volumes are not reused between different projects of the same name. | ||
|
||
## 12. How do I deploy my application to a local Minikube cluster? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The intent was to document how to use Docker daemon from minikube vs Docker on host.
For the FAQ section, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better structure overall, reads much more fluently than previously did.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Checklist
Description
Restructure "deploying" doc to be clearer
Related Issues
Related to #359
Fixes: appsody/appsody#885
Note: the
### Deploying your application through Docker Hub
and### Deploying your application to a custom registry
sections get fixed as part of #566